Skip to content

Comments

New Phase 3 TA Tip#357

Open
TwoLettuce wants to merge 5 commits intosoftwareconstruction240:mainfrom
TwoLettuce:main
Open

New Phase 3 TA Tip#357
TwoLettuce wants to merge 5 commits intosoftwareconstruction240:mainfrom
TwoLettuce:main

Conversation

@TwoLettuce
Copy link

Closes #356

Some students run into issues if they serialize their data using dependencies other than Gson, as those dependencies are not recognized by the autograder. This causes difficult issues for students when their code runs fine locally but won't compile or gives unexpected errors when run in the autograder, which is the reason I considered it requisite to add a TA tip to help students avoid this problem.

@mewilker
Copy link
Contributor

I think this PR looks good and is ready to merge!

Copy link
Contributor

@leesjensen leesjensen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Isn't the root of the problem is that the student doesn't return the correct objects from their endpoint and so Javlin suggests jackson to handle the serialization. Perhaps a better, or additional, tip would be if you get the jackson suggestion from javalin, to actually check what you are returning.

Added guidance on handling IntelliJ errors related to jackson dependency and autograder compilation issues.
@TwoLettuce
Copy link
Author

Isn't the root of the problem is that the student doesn't return the correct objects from their endpoint and so Javlin suggests jackson to handle the serialization. Perhaps a better, or additional, tip would be if you get the jackson suggestion from javalin, to actually check what you are returning.

Got it! I added an additional TA tip to help keep them from adding jackson to their projects in the first place and to instead check the return types.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add TA tip phase 3 to clarify that Jackson dependencies will not work with the autograder

3 participants